Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DPE-4416] Fetch charm libs to the latest LIBPATCH (update dp-libs to v36 to fix secrets issue) #475

Merged
merged 3 commits into from
May 27, 2024

Conversation

delgod
Copy link
Member

@delgod delgod commented May 27, 2024

Issue:
It is impossible to upgrade charm from the current 14/stable to 14/candidate due to the issue https://warthogs.atlassian.net/browse/DPE-4416

Solution:
Fix dp-library in revision 36 canonical/data-platform-libs#170
Update all libraries to the latest fixes/LIBPATCH.
Original issue is no longer reproducible and the new charm passed the internal testing.

@delgod delgod force-pushed the fix-tls-secrets branch from 7830e8c to 5329724 Compare May 27, 2024 08:18
@taurus-forever taurus-forever changed the title update dp-libs to v36 [DPE-4416] Fetch charm libs to the latest LIBPATCH (update dp-libs to v36 to fix secrets issue) May 27, 2024
@taurus-forever
Copy link
Contributor

I have reverted rolling-ops update due to requirements to tune unit-tests there: https://github.com/canonical/postgresql-operator/actions/runs/9251766314/job/25447979712 (not a scope to fix secrets on upgrade).

Copy link
Member

@lucasgameiroborges lucasgameiroborges left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM as it is, I'll investigate the issue with rolling-ops new version and why it breaks unit tests.

@delgod delgod merged commit ce40968 into main May 27, 2024
52 of 53 checks passed
@delgod delgod deleted the fix-tls-secrets branch May 27, 2024 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants